Skip to content

Setting(project): apps dev 서버 고정 port 지정 및 script 추가#270

Merged
constantly-dev merged 1 commit intodevelopfrom
refactor/#268/fix-dev-main-port-5173
Feb 24, 2026
Merged

Setting(project): apps dev 서버 고정 port 지정 및 script 추가#270
constantly-dev merged 1 commit intodevelopfrom
refactor/#268/fix-dev-main-port-5173

Conversation

@constantly-dev
Copy link
Member

@constantly-dev constantly-dev commented Feb 24, 2026 •
edited by coderabbitai bot
Loading

📌 Related Issues

관련된 Issue를 태그해주세요. (e.g. - close #25)

📄 Tasks

  • apps dev 서버 고정 port 지정 및 script 추가

⭐ PR Point (To Reviewer)

서버 환경 변수 설정으로 구글 소셜 로그인으로 redirect uri로 prod 버전과 localhost 5173 port를 지정해둔 상태예요.
따라서 개발할 대는 5173 port를 사용해야 로그인을 할 수 있는데, 모노레포 구조상 apps 하위 3개의 앱이 모두 매번 랜덤으로 포트가 열리는 상화이였어요.

로그인은 apps/client에서 사용하는데 5173 port가 아니면 매번 dev 서버를 다시 열어야 하는 불편함이 있었기에 이를 client에는 5173 port가 되도록, 이외 landing/extension도 5174/5175 port로 아예 고정을 했어요.

따라서 앞으로 개발할때는 이를 신경쓰지 않아도 괜찮아요. 추가로 앞으로 할 대부분의 개발은 apps/client의 비중이 높을텐데 이를 위해

"dev:client": "turbo run dev --filter=@pinback/client",

와 같은 script를 root에 추가했으니 필요에 따라 pnpm dev가 아닌 pnpm dev:client를 통해 개발을 진행하셔도 될 것 같아요.

Summary by CodeRabbit

  • Chores
    • 개발 서버 포트 구성을 업데이트했습니다. 클라이언트, 확장프로그램, 랜딩 페이지가 각각 별도의 포트에서 실행되도록 설정되었습니다.
    • 클라이언트 개발 환경용 새로운 개발 스크립트를 추가했습니다.

@vercel
Copy link

vercel bot commented Feb 24, 2026

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
pinback-client-client Ready Ready Preview, Comment Feb 24, 2026 6:47am
pinback-client-landing Ready Ready Preview, Comment Feb 24, 2026 6:47am

@github-actions github-actions bot added the refactor 코드 리팩토링 label Feb 24, 2026
@coderabbitai
Copy link

coderabbitai bot commented Feb 24, 2026 •
edited
Loading

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between 17daed1 and c045f5e.

📒 Files selected for processing (4)
  • apps/client/vite.config.ts
  • apps/extension/vite.config.ts
  • apps/landing/vite.config.ts
  • package.json

Walkthrough

세 개의 Vite 설정 파일(클라이언트, 확장, 랜딩)에 서버 포트 설정을 추가했습니다(각각 5173, 5175, 5174). 동시에 package.json에 클라이언트 전용 개발 스크립트를 추가하여 개발 환경 포트 바인딩을 명시화했습니다.

Changes

Cohort / File(s) Summary
Vite 개발 서버 포트 설정
apps/client/vite.config.ts, apps/extension/vite.config.ts, apps/landing/vite.config.ts		 각 앱별로 서버 설정 블록 추가: 클라이언트(포트 5173), 확장(포트 5175), 랜딩(포트 5174). strictPort를 true로 설정하여 포트 자동 증가 방지.
개발 스크립트 추가
package.json		 새로운 npm 스크립트 "dev:client" 추가: 클라이언트 패키지만 선별적으로 개발 서버 실행.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~10 minutes

Possibly related PRs

  • #268: 개발 환경 메인 포트를 5173으로 고정하는 스크립트 수정과 직접 관련된 설정 변경입니다.

Suggested labels

setting

Suggested reviewers

  • jllee000

Poem

🐰 포트가 떴네, 5173, 5175, 5174
세 길의 갈림길에서 각각의 집을 찾네
꼼꼼히 정해진 포트, 혼란 없이
개발의 길이 한결 맑아진다
토끼가 축하하며 발을 구르네! 🌟

🚥 Pre-merge checks | ✅ 4 | ❌ 1

❌ Failed checks (1 warning)

Check name Status Explanation Resolution
Linked Issues check ⚠️ Warning 코드 변경 사항이 #268의 개발 포트 5173 고정 목표를 충족합니다. #25는 Progress 컴포넌트로 이 PR과 무관합니다. #25는 이 PR과 무관한 이슈입니다. #268만 연결하거나 PR 목표를 명확히 해주세요.
✅ Passed checks (4 passed)
Check name Status Explanation
Title check ✅ Passed PR 제목이 변경 사항의 핵심인 고정된 포트 지정 및 스크립트 추가를 명확히 나타내고 있습니다.
Description check ✅ Passed PR 설명이 필수 섹션(Related Issues, Tasks, PR Point)을 모두 포함하고 있으며 충분한 맥락을 제공합니다.
Out of Scope Changes check ✅ Passed 모든 변경 사항(vite.config.ts 포트 고정 및 package.json 스크립트)이 #268의 목표와 일치합니다.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing Touches
  • 📝 Generate docstrings (stacked PR)
  • 📝 Generate docstrings (commit on current branch)
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch refactor/#268/fix-dev-main-port-5173

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@github-actions
Copy link

github-actions bot commented Feb 24, 2026

✅ Storybook chromatic 배포 확인:
🐿️ storybook

jjangminii
jjangminii approved these changes Feb 24, 2026
View reviewed changes
Copy link
Collaborator

@jjangminii jjangminii left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

이렇게 고정할 수 있네요-! 👍

@constantly-dev constantly-dev merged commit 8df6910 into develop Feb 24, 2026
11 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@jjangminii jjangminii jjangminii approved these changes

@jllee000 jllee000 Awaiting requested review from jllee000

Assignees

@constantly-dev constantly-dev

Labels

refactor 코드 리팩토링

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

[Refactor] fix dev main port 5173 (script)

2 participants

@constantly-dev @jjangminii